Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use nodejs version 18 instead of version 14 in when building test docker image #2951

Merged

Conversation

jorund1
Copy link
Collaborator

@jorund1 jorund1 commented Aug 19, 2024

why: nodejs version 14 gives a deprecation warning when building the test docker image. Version 18 is the oldest nodejs version that is not deprecated.

why: nodejs version 14 gives a deprecation warning when building the
docker image from test dockerfile. Version 18 is the oldest nodejs
version that is not deprecated.
@CLAassistant
Copy link

CLAassistant commented Aug 19, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

codecov bot commented Aug 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.35%. Comparing base (ea42b94) to head (762f195).
Report is 20 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2951   +/-   ##
=======================================
  Coverage   56.35%   56.35%           
=======================================
  Files         603      603           
  Lines       43849    43849           
  Branches       48       48           
=======================================
  Hits        24713    24713           
  Misses      19124    19124           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@lunkwill42 lunkwill42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you.

Legally speaking, it doesn't matter, since you're an employee of Sikt, but until you sign the CLA, the bot will keep nagging you on every PR until you do - so please do before we merge :)

@lunkwill42 lunkwill42 merged commit 7e0786c into Uninett:master Sep 4, 2024
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants